Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

flowint: add isnotset support (70x backport) - v1 #12242

Closed
wants to merge 2 commits into from

Conversation

jufajardini
Copy link
Contributor

Similar keywords use isnotset, while flowint only accepted notset Opted to change the code, not only the regex, to keep the underlying code also following the same patterns.

Task #7426

(cherry picked from commit 6e4a501)

Link to ticket: https://redmine.openinfosecfoundation.org/issues/
https://redmine.openinfosecfoundation.org/issues/7427

Describe changes:

  • Mostly a clean cherry-pick of 6e4a501
  • clang formatting changes

Provide values to any of the below to override the defaults.

SV_BRANCH=OISF/suricata-verify#2167

Similar keywords use `isnotset`, while `flowint` only accepted `notset`
Opted to change the code, not only the regex, to keep the underlying
code also following the same patterns.

Task OISF#7426

(cherry picked from commit 6e4a501)
Copy link

codecov bot commented Dec 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.18%. Comparing base (825eadf) to head (994a5a9).
Report is 6 commits behind head on main-7.0.x.

Additional details and impacted files
@@              Coverage Diff               @@
##           main-7.0.x   #12242      +/-   ##
==============================================
+ Coverage       83.17%   83.18%   +0.01%     
==============================================
  Files             922      922              
  Lines          260858   260858              
==============================================
+ Hits           216979   217007      +28     
+ Misses          43879    43851      -28     
Flag Coverage Δ
fuzzcorpus 64.17% <92.30%> (+0.02%) ⬆️
suricata-verify 63.37% <100.00%> (-0.03%) ⬇️
unittests 62.38% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 23711

@victorjulien
Copy link
Member

Merged in #12244, thanks!

@jufajardini jufajardini deleted the 7426-70x-7427/v1 branch December 9, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants